Skip to content

chore: sync skills CLI pin to 1.5.13#239

Merged
ryota-murakami merged 1 commit into
mainfrom
codex/sync-skills-cli-1-5-13
Jun 24, 2026
Merged

chore: sync skills CLI pin to 1.5.13#239
ryota-murakami merged 1 commit into
mainfrom
codex/sync-skills-cli-1-5-13

Conversation

@ryota-murakami

@ryota-murakami ryota-murakami commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • bump SKILLS_CLI_VERSION and the marketplace install regression pin to skills CLI 1.5.13
  • confirm upstream v1.5.13 agent definitions still resolve to 68 app-supported agents after exclusions
  • document the sync run and ignore local .agents skill copies in ESLint

Validation

  • node .agents/skills/skills-cli-sync/reconcile-agents.mjs
  • npx prettier --check README.md SPEC.md CLAUDE.md eslint.config.mjs .claude/skills/skills-cli-sync/SKILL.md .agents/skills/skills-cli-sync/SKILL.md
  • pnpm validate
  • pnpm test:e2e

Summary by CodeRabbit

  • 新機能

    • Skills CLI の参照バージョンを 1.5.13 に更新しました。
    • インストール時の検証内容を最新のバージョンに合わせて更新しました。
  • テスト

    • インストール回帰テストを最新バージョンに追従させました。
  • その他

    • 一部の非対象ファイルを静的チェックの対象外にしました。

Bump SKILLS_CLI_VERSION and the marketplace install regression pin from 1.5.11 to 1.5.13.

Confirmed upstream v1.5.13 has 72 agent names; app definitions remain at 68 because eve, promptscript, universal, and zenflow are excluded by the sync rules. No id/cliId migrations, display-name rebrands, or UNIVERSAL_AGENT_IDS changes.

Document the run in the skills-cli-sync skill and ignore local .agents skill copies in ESLint so pnpm validate stays green in this repo.
@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
skills-desktop Ready Ready Preview, Comment Jun 24, 2026 11:17am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 5404a6e1-88d6-4e84-aca4-ed6752fa322b

📥 Commits

Reviewing files that changed from the base of the PR and between 054929a and d30aeeb.

📒 Files selected for processing (5)
  • .claude/skills/skills-cli-sync/SKILL.md
  • CLAUDE.md
  • e2e/spec/marketplace-install-regression.e2e.ts
  • eslint.config.mjs
  • src/shared/constants.ts

Walkthrough

SKILLS_CLI_VERSION1.5.11 から 1.5.13 に更新し、src/shared/constants.tsCLAUDE.md・e2e テストの期待文字列・SKILL.md の実行履歴を同期。eslint.config.mjs.agents/** の除外ルールを追加。

Changes

Skills CLI バージョンピン更新

Layer / File(s) Summary
SKILLS_CLI_VERSION 定数更新と参照ドキュメント
src/shared/constants.ts, CLAUDE.md
SKILLS_CLI_VERSION'1.5.11''1.5.13' に変更し、CLAUDE.md のピン値記載を同期。
e2e アサーション更新と ESLint 除外設定
e2e/spec/marketplace-install-regression.e2e.ts, eslint.config.mjs
fake npx marker ファイルの期待文字列を skills@1.5.13 に更新。ignores.agents/** を追加。
SKILL.md 実行履歴追記
.claude/skills/skills-cli-sync/SKILL.md
v1.5.11 → v1.5.13 の Latest run 結果(除外理由・ゲート成否)を追記。

推定コードレビュー工数

🎯 1 (Trivial) | ⏱️ ~3 minutes

関連 PR

  • laststance/skills-desktop#215: 同じく SKILLS_CLI_VERSIONmarketplace-install-regression.e2e.ts の期待バージョンを更新しており、今回の変更と直接同系統のパターン。
  • laststance/skills-desktop#204: SKILLS_CLI_VERSION1.5.5 → 1.5.10 にバンプした先行 PR で、変更箇所が完全に一致。
  • laststance/skills-desktop#82: SKILLS_CLI_VERSION 定数の導入元 PR。npx skills@${SKILLS_CLI_VERSION} スポーン実装と e2e 期待値の原点。
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Skills CLI のピン更新と関連ファイルの同期内容を正しく要約しており、変更の主旨に一致しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/sync-skills-cli-1-5-13

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.86%. Comparing base (054929a) to head (d30aeeb).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #239   +/-   ##
=======================================
  Coverage   95.86%   95.86%           
=======================================
  Files         190      190           
  Lines        5726     5726           
  Branches     1297     1297           
=======================================
  Hits         5489     5489           
  Misses          5        5           
  Partials      232      232           
Flag Coverage Δ
unittests 95.86% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ryota-murakami ryota-murakami merged commit 3b86c65 into main Jun 24, 2026
11 checks passed
@ryota-murakami ryota-murakami deleted the codex/sync-skills-cli-1-5-13 branch June 24, 2026 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants